Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: expose the override data to the request events #2241

Merged
merged 3 commits into from
May 27, 2024

Conversation

Lightning00Blade
Copy link
Collaborator

@Lightning00Blade Lightning00Blade commented May 24, 2024

This PR make sure that updates via interception to the Request are properly show in the request property of responseStarted and responseCompleted events.
This should only be applicable to the request part as we get the update data from the responseRecived CDP events.

#2035

@Lightning00Blade Lightning00Blade changed the title fix: expose the override data to the events fix: expose the override data to the reqjes events May 24, 2024
@Lightning00Blade Lightning00Blade changed the title fix: expose the override data to the reqjes events fix: expose the override data to the request events May 24, 2024
@Lightning00Blade Lightning00Blade added the update-expectations Update WPT expectations label May 24, 2024
@Lightning00Blade Lightning00Blade marked this pull request as ready for review May 27, 2024 10:47
@Lightning00Blade Lightning00Blade merged commit 947bb8e into main May 27, 2024
48 checks passed
@Lightning00Blade Lightning00Blade deleted the fix-overrides-request-data branch May 27, 2024 11:06
Lightning00Blade pushed a commit that referenced this pull request May 31, 2024
🤖 I have created a release *beep* *boop*
---


##
[0.5.20](chromium-bidi-v0.5.19...chromium-bidi-v0.5.20)
(2024-05-31)


### Features

* send `browsingContext.contextCreated` event while subscribing
([#2255](#2255))
([592c839](592c839))
* support document as `startNodes` in `browsingContext.locateNodes`
([#2218](#2218))
([ad7318f](ad7318f))
* support for graphemes in key input
([#2207](#2207))
([8e3a6c0](8e3a6c0))


### Bug Fixes

* added missing input transformations
([#2186](#2186))
([ea48dc2](ea48dc2))
* css locator should allow nodes to be start nodes
([#2195](#2195))
([4a361a5](4a361a5))
* errors for `input.setFile`
([#2232](#2232))
([49e3712](49e3712))
* expose the override data to the request events
([#2241](#2241))
([947bb8e](947bb8e))
* report correct value for DefaultValue in UserPrompt
([#2228](#2228))
([df5ebf6](df5ebf6))
* screenshot taken in scrolled viewport origin
([#2161](#2161))
([b3c57c8](b3c57c8))
* throw invalid argument for header
([#2246](#2246))
([664d043](664d043))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

Co-authored-by: release-please[bot] <55107282+release-please[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
update-expectations Update WPT expectations
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants